Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial work on authentication #844

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

VictorDelCampo
Copy link
Member

RISDEV-5805

@VictorDelCampo VictorDelCampo added the do not merge For sharing prototypes or ideas that are not intended for merging label Dec 16, 2024
VictorDelCampo and others added 11 commits December 16, 2024 14:03
…805_spring_security_oauth2

# Conflicts:
#	backend/gradle/libs.versions.toml
RISDEV-5805
We use a service called localhost to be able to address the keycloak
service as "localhost:8443" in the application.yaml of our application.
Using "keycloak:8443" is not possible as spring would then redirect the
user to "keycloak:8443" to login, but that is a host that is not
available on the host system. The spring boot oauth implementation is
very restrictive on the issuer and therefore creates errors if the hosts
used by it and the user to connect to keycloak differ.

See also:
 - spring-projects/spring-security#14633
 - keycloak/keycloak#29783
 - keycloak/keycloak#24252
 - https://medium.com/@kostapchuk/integrating-keycloak-with-spring-boot-in-a-dockerized-environment-813eab1f140c

RISDEV-5805
RISDEV-5805
One for the full application and one for just the services. We need
different network configurations for both.

RISDEV-5805
Copy link

sonarqubecloud bot commented Jan 7, 2025

Copy link

sonarqubecloud bot commented Jan 7, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
45.5% Coverage on New Code (required ≥ 80%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge For sharing prototypes or ideas that are not intended for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants